Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@rhc54
Copy link

@rhc54 rhc54 commented May 3, 2016

(cherry picked from commit open-mpi/ompi@29bc24b)

… zero fields as this can be a problem for PSM

(cherry picked from commit 29bc24b)
@rhc54 rhc54 added this to the v2.0.0 milestone May 3, 2016
@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1610/ for details.

@hppritcha
Copy link
Member

@jsquyres this fixes omnipath problems we were seeing with our clusters. I suggest merging after CI completes.

@hppritcha
Copy link
Member

this fixes the 1 rank problem I was seeing on omnipath. 👍
It doesn't fix the multi-rank on one node problem - which was expected.

@hppritcha
Copy link
Member

@jsquyres good to go

@rhc54
Copy link
Author

rhc54 commented May 4, 2016

@hppritcha Are you then still blocked on going into production? Or do you have a temporary path forward until the complete fix is released?

@hppritcha
Copy link
Member

@rhc54 the sysadmins applied that patch from open-mpi/ompi#1578 and rebuilt the ompi 1.10. That suffices for them now. We do need this fix once the psm2 is fixed and we can go back to unpatched ompi. So no, not blocked.

@hppritcha
Copy link
Member

To be clearer, this fix should go in to 1.10.3 and 2.0.0.

@jsquyres jsquyres merged commit 60bc3d7 into open-mpi:v2.x May 5, 2016
@matcabral
Copy link
Contributor

fixes open-mpi/ompi#1559 epid=0 issue

@matcabral
Copy link
Contributor

bot:milestone:v2.0.0

@ompiteam-bot ompiteam-bot modified the milestones: v2.0.0, v1.10.3 May 10, 2016
@matcabral
Copy link
Contributor

bot:milestone:v1.10.3

@ompiteam-bot ompiteam-bot modified the milestones: v1.10.3, v2.0.0 May 24, 2016
@matcabral
Copy link
Contributor

bot:milestone:v2.0.0

@ompiteam-bot ompiteam-bot modified the milestones: v2.0.0, v1.10.3 May 25, 2016
@rhc54 rhc54 deleted the cmr20/psm2 branch May 25, 2016 00:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants